-
Notifications
You must be signed in to change notification settings - Fork 610
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to Xcode 11.2 #364
Update to Xcode 11.2 #364
Conversation
It looks like the build on TravisCI is failing. I only had a cursory look at the output and couldn't immediately see what's wrong, but this needs fixing. I'm also not sure about mixing a number of different concerns in this single PR, but I will comment on them individually. |
Looks like the Travis build failed for this particular run due to an issue in Apple's simulator control. My build of the same commit succeeded: https://travis-ci.com/madsolar8582/ocmock/builds/90191297 |
Good news, I re-ran the build manually and it passed now. |
Thanks Erik. I merged your current changes onto this branch. |
To be honest, I'd feel better to wait until Xcode 11 has left beta and is generally available. For future PRs: it was slightly confusing to see a PR with the title "Update to Xcode 11" and a date of November 2018. For a moment I thought I had missed something. Would be good open a new PR instead of repurposing an old one. |
Thank you for submitting these PR's. It seems you're always a step ahead of what's available through normal channels. I will now update to Xcode 11.1, which is, according to the AppStore, the current version. |
Description of Changes
Update CI image to Xcode 11 and apply recommended settings.
Motivation and Context
Migrating to latest Xcode to ensure compatibility and apply recommended settings to leverage best practices and leverage new quality checks.
How Has This Been Tested?
Local and CI successful builds.